Skip to content

fix(example-web): repair docker config and update deps#6342

Merged
overbalance merged 3 commits intoopen-telemetry:mainfrom
embrace-io:overbalance/repair-web-example
Jan 27, 2026
Merged

fix(example-web): repair docker config and update deps#6342
overbalance merged 3 commits intoopen-telemetry:mainfrom
embrace-io:overbalance/repair-web-example

Conversation

@overbalance
Copy link
Copy Markdown
Contributor

@overbalance overbalance commented Jan 26, 2026 •
edited
Loading

Which problem is this PR solving?

The opentelemetry-web example was broken due to outdated Docker configuration and dependencies:

  • Collector config had invalid sending_queue: batch: syntax
  • Collector was binding to 127.0.0.1 (localhost only inside container), making it inaccessible from the host
  • Collector image version 0.53.0 didn't support modern OTLP features (e.g., flags field in spans)

Short description of the changes

  • Fix collector-config.yaml: add explicit endpoint bindings (0.0.0.0:4317/4318) and proper batch processor definition
  • Update docker-compose.yaml: use latest collector image, remove deprecated version field
  • Update package.json: bump babel-loader to ^10.0.0, add --open flag to start script

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

  • Started Docker services with npm run docker:start
  • Started webpack dev server with npm start
  • Opened http://localhost:8090/fetch/ and clicked "Test" button
  • Verified traces are sent successfully (network requests to /v1/traces return 200, no console errors)

Checklist:

  • Followed the style guidelines of this project

@overbalance overbalance requested review from a team as code owners January 26, 2026 22:47
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 26, 2026 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.59%. Comparing base (29b32b5) to head (abedf9c).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main    #6342   +/-   ##
=======================================
  Coverage   95.59%   95.59%           
=======================================
  Files         314      314           
  Lines        9596     9596           
  Branches     2221     2221           
=======================================
  Hits         9173     9173           
  Misses        423      423
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@overbalance overbalance enabled auto-merge January 26, 2026 22:53
@overbalance overbalance changed the title fix(example-web): update docker and deps fix(example-web): repair docker config and update deps Jan 26, 2026
@overbalance overbalance mentioned this pull request Jan 27, 2026
Closed
1 task
@overbalance overbalance added this pull request to the merge queue Jan 27, 2026
Merged via the queue into open-telemetry:main with commit a148c77 Jan 27, 2026
27 checks passed
@overbalance overbalance deleted the overbalance/repair-web-example branch January 27, 2026 09:12
@otelbot-js otelbot-js Bot mentioned this pull request Feb 12, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@david-luna david-luna david-luna approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@overbalance @david-luna